Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New scale_segments() function #9570

Merged
merged 2 commits into from
Sep 24, 2022
Merged

New scale_segments() function #9570

merged 2 commits into from
Sep 24, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 24, 2022

Prepare for scale_segments

@AyushExel @Laughing-q FYI preparing to introduce a scale_segments() function to output mask outlines when with --save-txt etc.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Refactor of bounding box scaling functions and improved clipping in YOLOv5.

📊 Key Changes

  • Renamed scale_coords to scale_boxes, making the purpose clearer and more specific.
  • Renamed clip_coords to clip_boxes, aligning with the new bounding box-focused terminology.
  • Updated all function calls and references across the codebase to match the new names.
  • Enhanced the functions to handle bounding box scaling and clipping more efficiently.

🎯 Purpose & Impact

  • Clearer Codebase: The new naming conventions improve readability and make the codebase more intuitive.
  • Better Performance: Enhanced functions may lead to improvements in bounding box processing speed and accuracy.
  • Maintainability: With this cleanup, future maintenance and updates to the bounding box handling should be simplified.

With these changes, both developers and users can expect a more streamlined experience when working with bounding boxes in the YOLOv5 framework. 📦🚀

@glenn-jocher glenn-jocher changed the title Rename scale_coords to scale_boxes Rename scale_coords() to scale_boxes() Sep 24, 2022
@glenn-jocher glenn-jocher changed the title Rename scale_coords() to scale_boxes() New scale_segments() function Sep 24, 2022
@glenn-jocher glenn-jocher merged commit c8e5230 into master Sep 24, 2022
@glenn-jocher glenn-jocher deleted the update/scale_coords branch September 24, 2022 14:02
@AyushExel
Copy link
Contributor

OKay! got it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants